Skip to content

Comments

update getting started#358

Open
hkamm123 wants to merge 2 commits intosoftwareconstruction240:mainfrom
hkamm123:hkamm123-update-getting-started
Open

update getting started#358
hkamm123 wants to merge 2 commits intosoftwareconstruction240:mainfrom
hkamm123:hkamm123-update-getting-started

Conversation

@hkamm123
Copy link
Contributor

@hkamm123 hkamm123 commented Feb 20, 2026

Updates instruction information about server directory structure in response to server and client main classes being renamed and moved into packages. Also fixes minor grammar errors (commas, etc.) in phase 3 getting started document.

Copy link
Contributor

@Maillman Maillman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good! Let's get this merged in!

Once you have completed **all** of the previous steps, you should be able to launch your server and access a testing HTML page. This is a simple frontend that was made to help with basic testing of your server endpoints.

Inside of `server/src/main/java/Main.java` in the main method, replace the code that is there with the creation of the Server object. Then call the server's run method. The run method needs the port you will run your server on, which typically for testing is 8080.
Inside of `server/src/main/java/ServerMain.java` in the main method, replace the code that is there with the creation of the Server object. Then call the server's run method. The run method needs the port you will run your server on, which is typically 8080 for testing.
Copy link
Contributor

@jerodw jerodw Feb 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this correct or should it be server/src/main/java/server/ServerMain.java

If the code below is correct, its server/ServerMain.java

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants